From: Georges Basile Stavracas Neto Date: Wed, 10 Oct 2018 20:46:26 +0000 (-0300) Subject: switcherPopup: Bind to the stage, not the monitor X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~897^2~1^2~66 X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=c0b561dd4af184893d838f13b2fe7782bc502827;p=gnome-shell.git switcherPopup: Bind to the stage, not the monitor The switcher popup is a large, mostly transparent actor that should cover all the clickable area of GNOME Shell. In Clutter terms, it should cover the whole stage. By binding it to the primary monitor, the Alt+Tab behavior becomes a bit inconsistent. For example, by not hiding when clicking at empty spaces at other monitors. Fix that by binding the SwitcherPopup to the whole stage, and not only the primary monitor. Fixes https://gitlab.gnome.org/GNOME/gnome-shell/issues/647 --- diff --git a/js/ui/switcherPopup.js b/js/ui/switcherPopup.js index 537a6af0c..1bef1af70 100644 --- a/js/ui/switcherPopup.js +++ b/js/ui/switcherPopup.js @@ -11,7 +11,6 @@ const Shell = imports.gi.Shell; const Signals = imports.signals; const St = imports.gi.St; -const Layout = imports.ui.layout; const Main = imports.ui.main; const Tweener = imports.ui.tweener; @@ -65,7 +64,10 @@ var SwitcherPopup = new Lang.Class({ this._initialDelayTimeoutId = 0; this._noModsTimeoutId = 0; - this.add_constraint(new Layout.MonitorConstraint({ primary: true })); + this.add_constraint(new Clutter.BindConstraint({ + source: global.stage, + coordinate: Clutter.BindCoordinate.ALL, + })); // Initially disable hover so we ignore the enter-event if // the switcher appears underneath the current pointer location